[clarity-wasm-tests] fix Unchecked(TypeError(UIntType, IntType)) #6204
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❌ Your project check has failed because the head coverage (20.49%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## feat/clarity-wasm-develop #6204 +/- ##
=============================================================
- Coverage 20.51% 20.49% -0.02%
=============================================================
Files 528 528
Lines 392169 392169
Branches 323 323
=============================================================
- Hits 80438 80387 -51
- Misses 311723 311774 +51
Partials 8 8
... and 22 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actually, I would here argue that the test wasn't very pretty to begin with. How about a refactor which would make it a lot prettier: instead of having two variables with contracts and expected, we could have a single list of tuple (contract, expected). This way, we have no problem using our if cfg!..., and the testing loop can stop using an index and iterate directly for (contract, expected) in list_of_stuffs {...}.
csgui
force-pushed
the
wasm-issue-677/unchecked-type-error
branch
from
ff4e695 to
f410a1a
Compare
csgui
force-pushed
the
wasm-issue-677/unchecked-type-error
branch
from
f410a1a to
83a58de
Compare
csgui
merged commit e8ec90d
into
stacks-network:feat/clarity-wasm-develop
fix stacks-network/clarity-wasm#677